Conversation
  DeepSource reviewed changes in the commit range For detailed review results, please see the PR on DeepSource ↗ PR Report Card
Code Review Summary
How are these analyzer statuses calculated?Administrators can configure which issue categories are reported and cause analysis to be marked as failed when detected. This helps prevent bad and insecure code from being introduced in the codebase. If you're an administrator, you can modify this in the repository's settings. |
1 similar comment
DeepSource reviewed changes in the commit range For detailed review results, please see the PR on DeepSource ↗ PR Report Card
Code Review Summary
How are these analyzer statuses calculated?Administrators can configure which issue categories are reported and cause analysis to be marked as failed when detected. This helps prevent bad and insecure code from being introduced in the codebase. If you're an administrator, you can modify this in the repository's settings. |
|
|
Overall Grade |
Security Reliability Complexity Hygiene |
Code Review Summary
| Analyzer | Status | Updated (UTC) | Details |
|---|---|---|---|
| JavaScript |  |
Mar 11, 2026 8:36p.m. | Review ↗ |
|
@manuel-rw hi what else do i need to do here? most of the workflows dont seem to be running, also what do i need to do for documentation? |
|
I approved workflows. Please submit a PR in the docs repo. We will review this soon. |
|
awesome stuff thanks. heres the docs PR homarr-labs/documentation#494 |
manuel-rw
left a comment
manuel-rw
left a comment
There was a problem hiding this comment.
@Meierschlumpf can you take a look as well?
|
@manuel-rw thanks for the comments, sorry it took me a few days to get to, I have made the suggested changes and updated branch from main again |
manuel-rw
left a comment
manuel-rw
left a comment
There was a problem hiding this comment.
lgtm, @Meierschlumpf can you take a look?
|
@StrandedTurtle please run format:fix again :) |
Meierschlumpf
left a comment
Meierschlumpf
left a comment
There was a problem hiding this comment.
I haven't take a look at the widget and integration types, but here a first feedback from my side
|
@Meierschlumpf think i covered it all, lmk what else i can do to make it prod ready, will have more time this weekend |
|
@Meierschlumpf is it all looking ready now? |
…earr-integration
…earr-integration
Meierschlumpf
left a comment
Meierschlumpf
left a comment
There was a problem hiding this comment.
After this it looks good to me, thanks for the contribution. I saw you also created a pull request for the documentation (homarr-labs/documentation#494) can you add the widget there as well with some screenshots (similar to the other widget documentation pages)
|
@Meierschlumpf think thats done, will do the docs now |
|
@Meierschlumpf docs are updated with widgets and screenshots |
3 participants
Homarr
Thank you for your contribution. Please ensure that your pull request meets the following pull request:
pnpm build, autofix withpnpm format:fix)devbranchx,y,ior any abbrevation)hiya, this is my first time contributing to a public/open source project so please bear with me. i think i have done all of the checks above, but if anything needs work please let me know and ill get on it ASAP. Thanks :)